-
-
Notifications
You must be signed in to change notification settings - Fork 17.8k
bitcoin: 30.1 -> 29.1 #477611
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
bitcoin: 30.1 -> 29.1 #477611
Conversation
This reverts commit c9ccd15. Note that willcl-ark.gpg is still in the list of checked signers, see the discussion in NixOS#476387
This reverts commit b8758c3.
roconnor
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I reviewed the diff but didn't test it.
|
While bitcoin-core/bitcoincore.org#1213 is still a draft, I think it's a priority of the project to get v30.2 released. I wonder if it's worth downgrading now and then upgrading to v30.2 ~next week instead of just upgrading to v30.2 once it's released? |
|
+1 to wait for 30.2 |
|
My view is that the package is currently broken; cached binaries could cause users to lose funds; and we don't actually know when 30.2 will be available. |
|
Binaries for v30.2 are now available: https://bitcoincore.org/bin/bitcoin-core-30.2/. |
This includes a fix for https://groups.google.com/g/bitcoindev/c/QsC_FWPNGr4/m/t6MSDT6pBQAJ I've disabled the signers that didn't sign this release before the binaries were published. Only the following signers signed in time: - Sjors - guggero - pinheadmz - fanquake - 0xb10c - achow101 - Emzy - hebasto This closes NixOS#477611 as we don't need to downgrade anymore.
|
Opened #478769 for v30.2 |
Closing |
This includes a fix for https://groups.google.com/g/bitcoindev/c/QsC_FWPNGr4/m/t6MSDT6pBQAJ I've disabled the signers that didn't sign this release before the binaries were published. Only the following signers signed in time: - Sjors - guggero - pinheadmz - fanquake - 0xb10c - achow101 - Emzy - hebasto This closes NixOS#477611 as we don't need to downgrade anymore.
See https://groups.google.com/g/bitcoindev/c/QsC_FWPNGr4/m/t6MSDT6pBQAJ
This undoes #476387 and #451606
Bitcoin 30.1 and 30.0 are affected by a rare bug in the wallet compound, which may result in a loss of funds. Rolling back to safety.
Note that code signer willcl-ark which we added in #476387 is still in the list. He has signed 29.1, so we should keep him.
I propose to use 29.1 and not update to 29.2, because Ava Chow mentioned in the mailing list, that version 30 will be fixed in 30.2 and we can update right to it.
Analyzing release notes for 30.0 and 30.1 I haven't found any downgrade risks. The only caveat is coinstat index rebuilding, but they have accounted for it in the release notes.
Things done
passthru.tests.nixpkgs-reviewon this PR. See nixpkgs-review usage../result/bin/.Add a 👍 reaction to pull requests you find important.